https://github.com/rht/eurace_unibi/pull/1/checks?check_run_id=1639089209

The GitHub Action build for this repo will be enabled as soon as this PR is merged. Currently, it only runs on my fork.

@svdhoog

The tests actually failed, and I observed, have failed for most of the recent PRs. Maybe there needs to be set -e? Fixing the test is outside the scope of this PR though.

@rht For what purpose would you like to use CI for this repo?

Note that the unittests for this repo are not up-to-date, so this explains why they fail.

Maybe there needs to be set -e?

If I am not mistaken, this is at the beginning of the install_flame.sh script?

@rht For what purpose would you like to use CI for this repo?

Just for assurance that there are no bugs creeping in through the changes.

Note that the unittests for this repo are not up-to-date, so this explains why they fail.

Oh I see. IIRC they used to work during the early days when this repo went public. So the mismatch is caused by the recent commits?

If I am not mistaken, this is at the beginning of the install_flame.sh script?

I saw that it is there in install_flame.sh, so the install can notify a failure, but it appears that set -e inside run_tests.sh is commented out.